Skip to content

fix: fixed login to use current language#1118

Merged
arpargo merged 2 commits intomainfrom
KER-422-login-language
Feb 21, 2025
Merged

fix: fixed login to use current language#1118
arpargo merged 2 commits intomainfrom
KER-422-login-language

Conversation

@arpargo
Copy link
Contributor

@arpargo arpargo commented Feb 20, 2025

KK used to only direct to browser default language login page, that has now been fixed.
This required some changes to where the current language is stored and how it is used.
Further review on how the translations have been implemented would be beneficial.

@terovirtanen
Copy link
Contributor

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1118-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://kerrokantasi-pr1118-ui.dev.hel.ninja 😆🎉🎉🎉

mikkojamG
mikkojamG previously approved these changes Feb 21, 2025
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
63.6% Coverage on New Code (required ≥ 65%)

See analysis details on SonarQube Cloud

@terovirtanen
Copy link
Contributor

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1118-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://kerrokantasi-pr1118-ui.dev.hel.ninja 😆🎉🎉🎉

@arpargo arpargo merged commit 56161ba into main Feb 21, 2025
22 of 23 checks passed
@arpargo arpargo deleted the KER-422-login-language branch February 21, 2025 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants